Feat: Cadastrar, Renomear e Remover tipos de vegetações by JosueModesto · Pull Request #525 · utfpr/hcf-api

JosueModesto · 2026-09-04T19:36:23Z

Close #490

O que foi feito

Foi implementado o módulo de cadastro, atualização e remoção de tipos de vegetação na API v2, preservando a API legada sem alterações e seguindo o padrão de autenticação já utilizado pelo projeto.

Alterações principais

Implementação dos endpoints de vegetação na API v2:
- POST /api/v2/vegetacoes
- PUT /api/v2/vegetacoes/:vegetacaoId
- DELETE /api/v2/vegetacoes/:vegetacaoId
Validação de acesso de escrita com middleware específico para vegetações, respeitando o segredo JWT do projeto e os tipos de usuário permitidos.
Validação de nome vazio com retorno 400.
Validação de duplicidade de nome com retorno 409.
Validação de vegetacaoId inválido com retorno 400.
Retorno 404 quando o registro não existe.
Tratamento de conflito ao remover uma vegetação em uso em tombos com retorno 409.
Garantia de integridade no banco por meio de index único em LOWER(nome), além de tratamento compatível de erros de duplicidade e dependência em diferentes motores de banco.
Manutenção da API legacy /api sem alterações.

Cobertura de testes

Cadastro com sucesso
Cadastro com nome vazio
Cadastro com nome duplicado
Atualização de vegetação existente
Atualização com id inexistente
Atualização com id inválido
Atualização com nome duplicado
Remoção com sucesso
Remoção com vegetação em uso
Rejeição de token assinado com segredo inválido quando JWT_SECRET não está configurado

edvaldoszy · 2026-09-09T01:04:53Z


  return [
+    {
+      handlers: [


POST, PUT e DELETE montam só o controller em handlers. O legado POST /vegetacoes exige tokensMiddleware([CURADOR, OPERADOR]). País/estado v2 são só GET públicos e não justificam mutação anônima do catálogo. Os testes de escrita passam sem Authorization.

edvaldoszy · 2026-09-09T01:04:55Z

+    }
+  })
+
+  test('retorna 404 para id inexistente', async () => {


O PR pede nome vazio → 400, duplicidade → 409, vegetacaoId inválido → 400 e 404. Este arquivo só cobre sucesso, 404 e id abc. DELETE só cobre 204 e 409 em uso. No PUT, nome duplicado vira BadRequestError 400, não 409.

edvaldoszy · 2026-09-09T01:04:58Z

+      return Either.left(new CollectionError({ message: cause.message, cause }))
+    }
+  }
+


Unicidade é findByNome e depois insert/update. A tabela vegetacoes só tem PK, então duas requisições concorrentes com o mesmo nome passam no select e as duas gravam. O catch de unique/23505 não dispara sem constraint.

edvaldoszy · 2026-09-10T01:30:23Z

    updated_at timestamp with time zone DEFAULT CURRENT_TIMESTAMP NOT NULL
 );

+CREATE UNIQUE INDEX vegetacoes_nome_unique


O índice único em LOWER(nome) existe só no schema de integração. Não há migration em src/database e o model Sequelize de vegetacoes.nome não é unique. Em MySQL de produção o check-then-write continua sujeito a corrida e o 23505 do adapter não dispara.

edvaldoszy · 2026-09-10T01:30:24Z

+  const message = typeof error === 'object' && error !== null && 'message' in error ? String((error as { message?: unknown }).message) : ''
+  const constraint = typeof error === 'object' && error !== null && 'constraint' in error ? String((error as { constraint?: unknown }).constraint) : ''
+
+  return code === '23505'


Unique e FK estão mapeados com códigos/textos Postgres (23505, 23503, violates foreign key constraint). O app default é MySQL (ER_DUP_ENTRY 1062, ER_ROW_IS_REFERENCED_2 1451, foreign key constraint fails). Delete em uso no MySQL pode virar 500 em vez de 409 se error.constraint não vier preenchido.

edvaldoszy · 2026-09-10T01:30:25Z

+
+    if (!authorization || typeof authorization !== 'string' || !authorization.startsWith('Bearer ')) {
+      return new ForbiddenError({ message: 'Token de autenticação obrigatório' })
+    }


Token ausente ou vazio retorna 403, papel diferente de 1|2 retorna 403, expirado ou inválido retorna 401. Os testes de POST/PUT/DELETE só enviam token de curador no caminho feliz; esses ramos novos não estão cobertos.

edvaldoszy · 2026-09-10T01:30:26Z

+
+const ALLOWED_TIPOS_USUARIOS = new Set([1, 2])
+
+export class RequireVegetacaoWriteAccess implements RequestHandler {


RequireVegetacaoWriteAccess está no infinitivo inglês. Tipos novos devem usar 3ª pessoa do singular (RequiresVegetacaoWriteAccess) ou um verbo finito em português, sem misturar idiomas no nome.

edvaldoszy · 2026-09-10T01:30:27Z

+      }
+
+      const cause = error instanceof Error ? error : new Error(String(error))
+      return Either.left(new CollectionError({ message: cause.message, cause }))


O catch genérico devolve cause.message. Essa string chega no InternalServerError do controller e pode expor SQL no JSON de 500.

edvaldoszy · 2026-09-21T23:57:36Z

+
+const ALLOWED_TIPOS_USUARIOS = new Set([1, 2])
+
+export class ExigePermissaoEscritaVegetacao implements RequestHandler {


O arquivo RequerAcessoEscritaVegetacao.ts exporta ExigePermissaoEscritaVegetacao. O nome do arquivo deve coincidir com o tipo exportado: renomear para ExigePermissaoEscritaVegetacao.ts e ajustar o import em index.ts.

edvaldoszy · 2026-09-21T23:57:37Z

+  async handle(request: HttpRequest, next: NextHandler): Promise<HttpResponse | HttpError> {
+    const authorization = request.headers.Authorization ?? request.headers.authorization
+
+    if (!authorization || typeof authorization !== 'string' || !authorization.startsWith('Bearer ')) {


Token ausente ou vazio retorna 403, papel diferente de 1|2 retorna 403, expirado ou inválido retorna 401. Os testes de escrita cobrem o happy path e JWT_SECRET ausente; esses ramos ainda não estão cobertos.

edvaldoszy · 2026-09-21T23:57:38Z

+    }
+  })
+
+  test('retorna 404 para id inexistente', async () => {


Ainda faltam PUT com nome vazio (400) e DELETE com vegetacaoId inválido (400) e inexistente (404). Este arquivo cobre sucesso, 404, id inválido e duplicidade; remove-vegetacoes.test.ts só cobre 204 e 409 em uso.

Feat: Cadastrar, Renomear e Remover tipos de vegetações

867921f

JosueModesto requested review from edvaldoszy and a lite review from Copilot and removed request for Copilot September 4, 2026 19:37

Copilot started reviewing on behalf of JosueModesto September 4, 2026 19:38 View session

edvaldoszy reviewed Sep 9, 2026

View reviewed changes

Comment thread src/application/vegetacao/CadastraVegetacaoController.ts Outdated

edvaldoszy reviewed Sep 9, 2026

View reviewed changes

Comment thread src/application/vegetacao/RenomeiaVegetacaoController.ts Outdated

edvaldoszy reviewed Sep 9, 2026

View reviewed changes

Comment thread test/integration/vegetacao/cadastra-vegetacoes.test.ts Outdated

edvaldoszy requested changes Sep 9, 2026

View reviewed changes

JosueModesto added 2 commits September 9, 2026 21:01

fix: correções e adição de token

5e8c7aa

fix: add jsonwebtoken types

a8db337

edvaldoszy reviewed Sep 10, 2026

View reviewed changes

Comment thread src/application/vegetacao/RequireVegetacaoWriteAccess.ts Outdated

edvaldoszy reviewed Sep 10, 2026

View reviewed changes

edvaldoszy requested changes Sep 10, 2026

View reviewed changes

JosueModesto added 2 commits September 16, 2026 17:04

Correções da review

b334e25

correções test

a14b7ef

edvaldoszy reviewed Sep 21, 2026

View reviewed changes

edvaldoszy requested changes Sep 21, 2026

View reviewed changes

Navigation Menu

Feat: Cadastrar, Renomear e Remover tipos de vegetações - #525

Feat: Cadastrar, Renomear e Remover tipos de vegetações#525
JosueModesto wants to merge 5 commits into
developmentfrom
490-cadastrar-renomear-remover-tipos-vegetacoes

JosueModesto commented Sep 4, 2026 •

edited

Loading

Uh oh!

edvaldoszy Sep 9, 2026

Uh oh!

Uh oh!

edvaldoszy Sep 9, 2026

Uh oh!

Uh oh!

edvaldoszy Sep 9, 2026

Uh oh!

Uh oh!

Uh oh!

edvaldoszy Sep 10, 2026

Uh oh!

edvaldoszy Sep 10, 2026

Uh oh!

edvaldoszy Sep 10, 2026

Uh oh!

edvaldoszy Sep 10, 2026

Uh oh!

edvaldoszy Sep 10, 2026

Uh oh!

edvaldoszy Sep 21, 2026

Uh oh!

edvaldoszy Sep 21, 2026

Uh oh!

edvaldoszy Sep 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants


		const ALLOWED_TIPOS_USUARIOS = new Set([1, 2])

		export class RequireVegetacaoWriteAccess implements RequestHandler {


		const ALLOWED_TIPOS_USUARIOS = new Set([1, 2])

		export class ExigePermissaoEscritaVegetacao implements RequestHandler {

Conversation

JosueModesto commented Sep 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

O que foi feito

Alterações principais

Cobertura de testes

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

JosueModesto commented Sep 4, 2026 •

edited

Loading